Skip to content

Headlines filter UI logic enhance #19

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 16 commits into from
May 29, 2025
Merged

Conversation

fulleni
Copy link
Member

@fulleni fulleni commented May 29, 2025

Description

Type of Change

  • [ x] ✨ New feature (non-breaking change which adds functionality)
  • 🛠️ Bug fix (non-breaking change which fixes an issue)
  • ❌ Breaking change (fix or feature that would cause existing functionality to change)
  • [ x] 🧹 Code refactor
  • ✅ Build configuration change
  • 📝 Documentation
  • 🗑️ Chore

fulleni added 16 commits May 30, 2025 00:06
- Removed unused routes
- Simplified routing logic
- Removed country filter page
- Removed country filter bloc
- Removed unused filter
- Improved readability
- Removed country filter
- Simplified filter logic
- Remove status enums
- Remove cursor and error
- Add country and source type filters
- Added new filter events
- Toggles for country, type, source
- Event to clear filters
- Implemented filter by country/type
- Added data loading state
- Added clear filters functionality
- Removed unused Country model import
- Removed country filter parameter
- Simplified API request construction
- Add country and type filters
- Improve loading and error states
- Use BLoC for data loading
- Added filter loading text
- Added country/type labels
- Added source error messages
- Temporarily disable navigation
- Until new flow is defined
- Removed retry button
- Clarified empty state message
@fulleni fulleni merged commit 4f4fb6c into main May 29, 2025
0 of 2 checks passed
@fulleni fulleni deleted the headlines_filter_ui_logic_enhance branch May 29, 2025 23:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant